Skip to content

chore: wire restack UI, add Vitest, docs cleanup + dogfooding checklist#8

Closed
alejandro-nexcade wants to merge 3 commits into
alejandro/per-project-plansfrom
alejandro/followups
Closed

chore: wire restack UI, add Vitest, docs cleanup + dogfooding checklist#8
alejandro-nexcade wants to merge 3 commits into
alejandro/per-project-plansfrom
alejandro/followups

Conversation

@alejandro-nexcade

Copy link
Copy Markdown
Collaborator

Stacked on #7 (per-project plans). Merge #7 first; GitHub will retarget this to main.

  • feat(app): wire the existing restack_pr command into the UI — Restack button on stale reviews (DiffView header + card), disabled while the conflict-resolver runs. (Last dangling command wired.)
  • test(app): Vitest harness (jsdom + testing-library, typed Tauri mocks); 48 tests across ci.ts/shortcuts/store/StateFilter/ReviewCard/CiPanel; CI frontend job now runs vitest run.
  • docs: remove stale duplicate cockpit-docs/{CLAUDE,SPEC,IMPLEMENTATION_PLAN}.md; add cockpit-docs/DOGFOODING.md (manual checklist for the live-only paths: agent loop, plan fan-out, CI, LSP, links).

Verified: cargo check, npx tsc, npx vitest run (48/48), npm run build, npm ci — all green.

🤖 Generated with Claude Code

LifeLex and others added 3 commits July 1, 2026 17:37
Adds a Restack action (store.restackPr + button) shown only for stale reviews, in the DiffView header and the review card; disabled while a conflict-resolver agent runs.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
cockpit-docs/{CLAUDE,SPEC,IMPLEMENTATION_PLAN}.md were stale duplicates of the root docs (accidentally committed); remove them. Add cockpit-docs/DOGFOODING.md for the live-only verification paths.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
vitest + jsdom + testing-library; typed Tauri invoke/listen mocks; 48 tests across ci.ts, shortcuts, store, StateFilter, ReviewCard, CiPanel. CI frontend job now runs npx vitest run after tsc.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants